Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/add kind vm #18

Merged
merged 13 commits into from
Dec 10, 2024
Merged

Feature/add kind vm #18

merged 13 commits into from
Dec 10, 2024

Conversation

j-kali
Copy link
Contributor

@j-kali j-kali commented Nov 19, 2024

  1. Added definition of a pouta VM with a kind cluster to host a dev/test environment
  2. Converted k8s yaml's into a helm chart
  3. Adjusted the ansible playbook to work nicely with the dev/test environment

New collaborators:

	5E984B2D Jarosław Kalinowski (CSC) <[email protected]>
* Update outdated dependencies
* Split installation into multiple steps to aid debugging
* Remove replace ';' with '&&' to fail properly when anything fails
@j-kali j-kali self-assigned this Nov 19, 2024
@j-kali j-kali force-pushed the feature/add-kind-vm branch 2 times, most recently from 6a7383b to 1dcf74c Compare November 19, 2024 12:47
@j-kali j-kali marked this pull request as ready for review November 19, 2024 13:35
@j-kali j-kali force-pushed the feature/add-kind-vm branch from d54631b to 30437d8 Compare November 19, 2024 13:39
@j-kali j-kali force-pushed the feature/add-kind-vm branch 2 times, most recently from feea2b8 to 6a5af1c Compare November 26, 2024 11:22
@j-kali j-kali force-pushed the feature/add-kind-vm branch from 6a5af1c to 456930f Compare November 26, 2024 12:40
Copy link
Collaborator

@mmatthiesencsc mmatthiesencsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dear Jaroslaw, I finally reviewed all files. I don't know terraform and I am not claiming that I internalized everything, but thanks for this, it looks reasonably clear. One question, does this contain your latest local changes?

@j-kali
Copy link
Contributor Author

j-kali commented Dec 10, 2024

One question, does this contain your latest local changes?

Tree looks clean here so I think everything is commited and pushed 🤔

There is a good chance something is still specific to my setup, or something is not clear. But I think it will come out in due time and I will be around to help. I see no reason not to merge it at this stage 🤷

I can see in history that merge requests were merged using a "merge commit" as opposed to just fast forwarding, so I guess I will keep things consistent.

@j-kali j-kali merged commit 8e54635 into main Dec 10, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants